Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update CODEOWNERS #1805

Merged
merged 1 commit into from
Nov 7, 2022
Merged

Conversation

tongyu-microsoft
Copy link
Contributor

Update CODEOWNERS

Description

CODEOWNERS

Checklist

  • I have added screenshots above for all UI changes.
  • I have added e2e tests for all UI changes.
  • Documentation was updated if it was needed.

@tongyu-microsoft tongyu-microsoft enabled auto-merge (squash) November 7, 2022 18:58
@codecov-commenter
Copy link

codecov-commenter commented Nov 7, 2022

Codecov Report

Merging #1805 (cb06605) into main (c541255) will decrease coverage by 11.20%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1805       +/-   ##
===========================================
- Coverage   87.53%   76.32%   -11.21%     
===========================================
  Files          96       26       -70     
  Lines        5390      528     -4862     
===========================================
- Hits         4718      403     -4315     
+ Misses        672      125      -547     
Flag Coverage Δ
unittests 76.32% <ø> (-11.21%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ponsibleai/databalanceanalysis/balance_measures.py
...i/responsibleai/managers/counterfactual_manager.py
...is/erroranalysis/_internal/surrogate_error_tree.py
...ibleai/responsibleai/managers/explainer_manager.py
...nsibleai/databalanceanalysis/data_balance_utils.py
...bleai/responsibleai/_tools/causal/causal_config.py
erroranalysis/erroranalysis/_internal/utils.py
responsibleai/responsibleai/_interfaces.py
...nalysis/erroranalysis/_internal/version_checker.py
raiwidgets/raiwidgets/fairness_dashboard.py
... and 60 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@tongyu-microsoft tongyu-microsoft merged commit e3a4d87 into main Nov 7, 2022
@tongyu-microsoft tongyu-microsoft deleted the tongyu-microsoft/updateCodeOwner branch November 7, 2022 20:08
gaugup pushed a commit that referenced this pull request Nov 8, 2022
gaugup added a commit that referenced this pull request Nov 18, 2022
* Add postga build trigger (#1755) (#1756)

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Add model wrapper for wrapping predictions and test data (#1762)

* Add model wrapper for wrapping predictions and test data

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix failing tests

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Change description of cohort selection panel in Aggregate Feature Importance (#1770)

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Support cohort filtering of string target in rai_insights (#1771)

* Port tests

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix rai_insights

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Simplify tests in test_cohort_filter.py (#1772)

* Simply tests in test_cohort_filter.py

Signed-off-by: Gaurav Gupta <[email protected]>

* Change test name

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Move cohort re-labelling logic to Cohort.ts (#1773)

* Move cohort re-labelling logic to Cohort.ts

Signed-off-by: Gaurav Gupta <[email protected]>

* Lint fix

Signed-off-by: Gaurav Gupta <[email protected]>

* Lint fix

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Update requirements.txt (#1779)

* Refactor DataAnalysisTab into Chartview/DataAnalysisView and DataBalanceView (#1775)

* Refactor DataAnalysisTab into Chartview/DataAnalysisView and DataBalanceView

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix lint

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* [Highcharts] Enable csv download, xls download and view data table in highcharts (#1783)

* enable csv xls download in hightcharts

* localization

* Add new data APIs for compute_matrix() and compute_error_tree() (#1790)

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* update e2e visit function (#1792)

* Add rai_test_utils package (#1777)

* Add rai_test_utils package

Signed-off-by: Gaurav Gupta <[email protected]>

* Lint fix and namespace changes

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix duplicated cohorts not retaining the selection in combobox (#1785)

* fix and e2e

Signed-off-by: vinutha karanth <[email protected]>

* update filter

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>

* release raiwidgets and responsibleai 0.23.0 (#1787)

* fix release github action for raiwidgets and responsibleai by making it same as in gated build (#1788)

* Add e2e for dependency plot Avg of abs value option (#1768)

* fix dep plot

Signed-off-by: vinutha karanth <[email protected]>

* add e2e

Signed-off-by: vinutha karanth <[email protected]>

* lintfix

Signed-off-by: vinutha karanth <[email protected]>

* lintfix

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>

* update e2e visit function (#1792)

* Fix code review comments

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
Signed-off-by: vinutha karanth <[email protected]>
Co-authored-by: Vinutha Karanth <[email protected]>
Co-authored-by: Ilya Matiach <[email protected]>
Co-authored-by: tongy-msft <[email protected]>

* sort fix (#1798)

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>

* [LogarithmicScaling] Enable log scaling for data explorer, feature importance, counterfactual and casual (#1802)

* update checkbox to toggle in axis config panel

* support log toggle for counterfactual

* enable log scaling for feature importance

* enable log scaling for casual and data explorer

* address comment

* Update CODEOWNERS (#1805)

* Fix ‘Close button’ and ‘Spin button’ present in new cohort dialog does not have any title or aria-label (#1806)

* add aria label

Signed-off-by: vinutha karanth <[email protected]>

* address comment

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>

* Update erroranalysis tests to use new rai_test_utils package (#1800)

* Update erroranalysis tests to use new rai_test_utils package

Signed-off-by: Gaurav Gupta <[email protected]>

* Remove shap and xgboost from test requirements

Signed-off-by: Gaurav Gupta <[email protected]>

* Pin rai_test_utils

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix Heatmap tooltip theme (#1808)

* fix tooltip color

Signed-off-by: vinutha karanth <[email protected]>

* fix theme

Signed-off-by: vinutha karanth <[email protected]>

* fix

Signed-off-by: vinutha karanth <[email protected]>

* rev

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>

* Calculate RAI insights for first 5K samples (#1803)

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Add API for indicating large data scenario in ModelAssessmentContext (#1804)

* [WIP] Add APIs for indicating large data scenario in ModelAssessmentContext

Signed-off-by: Gaurav Gupta <[email protected]>

* Change API name

Signed-off-by: Gaurav Gupta <[email protected]>

* Add unit test

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix lint

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix lint again

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Add RAI version number to oss serialize result (#1812)

* Add RAI version number to oss serialize result

* update json file namg

* add test

Signed-off-by: Gaurav Gupta <[email protected]>
Signed-off-by: vinutha karanth <[email protected]>
Co-authored-by: tongy-msft <[email protected]>
Co-authored-by: Vinutha Karanth <[email protected]>
Co-authored-by: Ilya Matiach <[email protected]>
tongyu-microsoft added a commit that referenced this pull request Dec 7, 2022
…er, counterfactual and causal managers (#1824)

* Add initial code for supporting dropped features

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix causal path

Signed-off-by: Gaurav Gupta <[email protected]>

* Validate counerfactual scenario

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix UI build

Signed-off-by: Gaurav Gupta <[email protected]>

* Add postga build trigger (#1755) (#1756)

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Add model wrapper for wrapping predictions and test data (#1762)

* Add model wrapper for wrapping predictions and test data

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix failing tests

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Change description of cohort selection panel in Aggregate Feature Importance (#1770)

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Support cohort filtering of string target in rai_insights (#1771)

* Port tests

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix rai_insights

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Simplify tests in test_cohort_filter.py (#1772)

* Simply tests in test_cohort_filter.py

Signed-off-by: Gaurav Gupta <[email protected]>

* Change test name

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Move cohort re-labelling logic to Cohort.ts (#1773)

* Move cohort re-labelling logic to Cohort.ts

Signed-off-by: Gaurav Gupta <[email protected]>

* Lint fix

Signed-off-by: Gaurav Gupta <[email protected]>

* Lint fix

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Update requirements.txt (#1779)

* Refactor DataAnalysisTab into Chartview/DataAnalysisView and DataBalanceView (#1775)

* Refactor DataAnalysisTab into Chartview/DataAnalysisView and DataBalanceView

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix lint

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* [Highcharts] Enable csv download, xls download and view data table in highcharts (#1783)

* enable csv xls download in hightcharts

* localization

* Add new data APIs for compute_matrix() and compute_error_tree() (#1790)

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* update e2e visit function (#1792)

* Add rai_test_utils package (#1777)

* Add rai_test_utils package

Signed-off-by: Gaurav Gupta <[email protected]>

* Lint fix and namespace changes

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix duplicated cohorts not retaining the selection in combobox (#1785)

* fix and e2e

Signed-off-by: vinutha karanth <[email protected]>

* update filter

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>

* release raiwidgets and responsibleai 0.23.0 (#1787)

* fix release github action for raiwidgets and responsibleai by making it same as in gated build (#1788)

* Add e2e for dependency plot Avg of abs value option (#1768)

* fix dep plot

Signed-off-by: vinutha karanth <[email protected]>

* add e2e

Signed-off-by: vinutha karanth <[email protected]>

* lintfix

Signed-off-by: vinutha karanth <[email protected]>

* lintfix

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>

* update e2e visit function (#1792)

* Fix code review comments

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>
Signed-off-by: vinutha karanth <[email protected]>
Co-authored-by: Vinutha Karanth <[email protected]>
Co-authored-by: Ilya Matiach <[email protected]>
Co-authored-by: tongy-msft <[email protected]>

* sort fix (#1798)

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>

* [LogarithmicScaling] Enable log scaling for data explorer, feature importance, counterfactual and casual (#1802)

* update checkbox to toggle in axis config panel

* support log toggle for counterfactual

* enable log scaling for feature importance

* enable log scaling for casual and data explorer

* address comment

* Update CODEOWNERS (#1805)

* Fix ‘Close button’ and ‘Spin button’ present in new cohort dialog does not have any title or aria-label (#1806)

* add aria label

Signed-off-by: vinutha karanth <[email protected]>

* address comment

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>

* Update erroranalysis tests to use new rai_test_utils package (#1800)

* Update erroranalysis tests to use new rai_test_utils package

Signed-off-by: Gaurav Gupta <[email protected]>

* Remove shap and xgboost from test requirements

Signed-off-by: Gaurav Gupta <[email protected]>

* Pin rai_test_utils

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix Heatmap tooltip theme (#1808)

* fix tooltip color

Signed-off-by: vinutha karanth <[email protected]>

* fix theme

Signed-off-by: vinutha karanth <[email protected]>

* fix

Signed-off-by: vinutha karanth <[email protected]>

* rev

Signed-off-by: vinutha karanth <[email protected]>

Signed-off-by: vinutha karanth <[email protected]>

* Calculate RAI insights for first 5K samples (#1803)

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Add API for indicating large data scenario in ModelAssessmentContext (#1804)

* [WIP] Add APIs for indicating large data scenario in ModelAssessmentContext

Signed-off-by: Gaurav Gupta <[email protected]>

* Change API name

Signed-off-by: Gaurav Gupta <[email protected]>

* Add unit test

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix lint

Signed-off-by: Gaurav Gupta <[email protected]>

* Fix lint again

Signed-off-by: Gaurav Gupta <[email protected]>

Signed-off-by: Gaurav Gupta <[email protected]>

* Add RAI version number to oss serialize result (#1812)

* Add RAI version number to oss serialize result

* update json file namg

* add test

* enable dropped_features for EA and explainer

* enable EA and explainer for classification

* remove unused comments

* revert notebook change

* revert notebook change

* revert notebook change

* revert causal cf changes

* refactor getFeatureNamesAfterDrop  to a common method

* address comments on modelWrapper

* update comments

* update index.ts

* remove unused variable

* fix lint error

* fix noneType error

* address comments

* fix a test error

* support cf and causal

* hide dropped_features for cf and causal

* fix undefined feature_metadata error

* fix undefined feature_metadata error

* fix wrong cf predict value

* fix undefined feature_metadata

* fix undefined feature_metadata

* address comments

* address comments

* fix feature_names test error

* fix _feature_metadata undefined

* fix failing test on undefined model

Signed-off-by: Gaurav Gupta <[email protected]>
Signed-off-by: vinutha karanth <[email protected]>
Co-authored-by: Gaurav Gupta <[email protected]>
Co-authored-by: Gaurav Gupta <[email protected]>
Co-authored-by: Vinutha Karanth <[email protected]>
Co-authored-by: Ilya Matiach <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants